Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update rustls and release 1.3.1 #192

Merged
merged 1 commit into from
Apr 22, 2024
Merged

Update rustls and release 1.3.1 #192

merged 1 commit into from
Apr 22, 2024

Conversation

robin-nitrokey
Copy link
Member

No description provided.

@sosthene-nitrokey
Copy link
Contributor

I think we can ignore those recent warnings to release now.

@robin-nitrokey
Copy link
Member Author

Looking at the advisory on GitHub, only rustls servers are affected. This crate only uses rustls as a client. We can upgrade anyway, but I would not consider it urgent and would fix the CI first.

@sosthene-nitrokey
Copy link
Contributor

The RUSTSEC advisory is more generic: https://rustsec.org/advisories/RUSTSEC-2024-0336.html

If a close_notify alert is received during a handshake, complete_io does not terminate. Callers which do not call complete_io are not affected.

Is says handshake not just server, and complete_io is part of CommonConnection that is both used in sever code and client code.

@sosthene-nitrokey
Copy link
Contributor

The ureq devs make the same point, though complete_io is used in the ureq codebase: https://github.com/algesten/ureq/blob/f32a752cdbd9a9e2e9cca89b8df40512740f3523/src/rtls.rs#L108

@sosthene-nitrokey
Copy link
Contributor

The CI is fixed by #193

@robin-nitrokey
Copy link
Member Author

AFAIS the GitHub advisory is the original source and the RUSTSEC advisory is based on the GitHub advisory. I would assume that the details got lost in the process. Let’s just merge the clippy fix and then release the fix.

@robin-nitrokey robin-nitrokey merged commit 8842e3a into main Apr 22, 2024
3 checks passed
@robin-nitrokey robin-nitrokey deleted the update-rustls branch April 22, 2024 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants